-
Notifications
You must be signed in to change notification settings - Fork 1
chore: Adds release workflow and updates module name #5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
| @@ -1,4 +1,4 @@ | |||
| module go.mongodb.org/atlas-appservices | |||
| module github.com/mongodb-labs/go-client-mongodb-atlas-app-services | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updating module name to match the repo for easy installation otherwise a vanity URL setup would be required
| - "${signature}" | ||
| - "--detach-sign" | ||
| - "${artifact}" | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can add prerelease param is you want to support pre-releases, e.g.: https://github.com/mongodb-labs/atlas-cli-plugin-terraform/blob/153a3877c13a52101879f7f32150107a5210d4fa/.goreleaser.yaml#L57
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think that's required at this point. Since this will eventually be deprecated and I don't see much use-cases for pre-releases as this client is quite light anyway
Proposed changes
Tested release process in forked repo: https://github.com/maastha/go-client-mongodb-atlas-app-services/releases/tag/v0.0.2-pre
Jira ticket: CLOUDP-273338
Checklist
make fmtand formatted my codeFurther comments